feat!: remove Elastic Trace processor from EDOT Collector #10378
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This pull request does not have a backport label. Could you fix it @andrzej-stencel? 🙏
|
🔍 Preview links for changed docs |
It is replaced with the Elastic APM processor.
andrzej-stencel
force-pushed
the
replace-elastic-trace-processor-with-elastic-apm-processor
branch
from
e07e280 to
c1e9b4e
Compare
theletterf
requested changes
Oct 7, 2025
There was a problem hiding this comment.
Thanks!
Please revert the changes in components.md and custom-collector.md, since those docs are meant to show the latest stable release (9.1.5) and will update automatically upon 9.2 release.
I would also recommend to edit all the other sample configurations and backport (forwardport?) the changes to the cut 9.2 branch.
pierrehilbert
added
the
Team:Elastic-Agent-Control-Plane
|
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
andrzej-stencel
added a commit
to andrzej-stencel/elastic-agent
that referenced
this pull request
Oct 7, 2025
In anticipation of elastic#10378
Right, fixed in 56127ab. |
theletterf
approved these changes
Oct 7, 2025
💚 Build Succeeded
|
3 tasks
ycombinator
approved these changes
Oct 7, 2025
ChrsMark
approved these changes
Oct 8, 2025
mergify bot
pushed a commit
that referenced
this pull request
Oct 8, 2025
* feat!: remove Elastic Trace processor from EDOT Collector It is replaced with the Elastic APM processor. * fix: add Elastic APM processor to custom collector docs * revert changes in docs * fix component order in docs (cherry picked from commit 6c2491c)
andrzej-stencel
deleted the
replace-elastic-trace-processor-with-elastic-apm-processor
branch
|
We just had a zoom call with @andrzej-stencel and @AlexanderWert and concluded that removing this processor would be a breaking change, which we'd like to avoid. On the other hand, the code of the elastic trace processor is already removed from the collector components repo, so the last version depends on older upstream components, which makes bumping versions harder - so this would have not worked probably anyways as it was in this repo. Plan is that I'll try to create a small processor with the same name |
andrzej-stencel
added a commit
that referenced
this pull request
Oct 8, 2025
* sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v0\.135.0/\1 v0.137.0/' go.mod * sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v1\.41\.0/\1 v1.43.0/' go.mod * sed -i 's/\(github\.com\/open-telemetry\/opentelemetry\-collector\-contrib\/.*\) v0\.135\.0/\1 v0.137.0/' go.mod * go mod tidy * update Elastic components * remove Elastic Trace processor In anticipation of #10378 * go get go.opentelemetry.io/[email protected] * go mod tidy * add changelog entry
mergify bot
pushed a commit
that referenced
this pull request
Oct 8, 2025
* sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v0\.135.0/\1 v0.137.0/' go.mod * sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v1\.41\.0/\1 v1.43.0/' go.mod * sed -i 's/\(github\.com\/open-telemetry\/opentelemetry\-collector\-contrib\/.*\) v0\.135\.0/\1 v0.137.0/' go.mod * go mod tidy * update Elastic components * remove Elastic Trace processor In anticipation of #10378 * go get go.opentelemetry.io/[email protected] * go mod tidy * add changelog entry (cherry picked from commit 7ea323e) # Conflicts: # go.mod # go.sum # internal/pkg/otel/README.md
7 tasks
andrzej-stencel
added a commit
that referenced
this pull request
Oct 8, 2025
…10406) * feat!: remove Elastic Trace processor from EDOT Collector It is replaced with the Elastic APM processor. * fix: add Elastic APM processor to custom collector docs * revert changes in docs * fix component order in docs (cherry picked from commit 6c2491c) Co-authored-by: Andrzej Stencel <[email protected]>
andrzej-stencel
added a commit
that referenced
this pull request
Oct 8, 2025
* sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v0\.135.0/\1 v0.137.0/' go.mod * sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v1\.41\.0/\1 v1.43.0/' go.mod * sed -i 's/\(github\.com\/open-telemetry\/opentelemetry\-collector\-contrib\/.*\) v0\.135\.0/\1 v0.137.0/' go.mod * go mod tidy * update Elastic components * remove Elastic Trace processor In anticipation of #10378 * go get go.opentelemetry.io/[email protected] * go mod tidy * add changelog entry (cherry picked from commit 7ea323e) # Conflicts: # go.mod # go.sum # internal/pkg/otel/README.md Co-authored-by: Andrzej Stencel <[email protected]>
mergify bot
pushed a commit
that referenced
this pull request
Oct 14, 2025
* sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v0\.135.0/\1 v0.137.0/' go.mod * sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v1\.41\.0/\1 v1.43.0/' go.mod * sed -i 's/\(github\.com\/open-telemetry\/opentelemetry\-collector\-contrib\/.*\) v0\.135\.0/\1 v0.137.0/' go.mod * go mod tidy * update Elastic components * remove Elastic Trace processor In anticipation of #10378 * go get go.opentelemetry.io/[email protected] * go mod tidy * add changelog entry (cherry picked from commit 7ea323e) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md
3 tasks
pkoutsovasilis
pushed a commit
that referenced
this pull request
Oct 16, 2025
* sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v0\.135.0/\1 v0.137.0/' go.mod * sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v1\.41\.0/\1 v1.43.0/' go.mod * sed -i 's/\(github\.com\/open-telemetry\/opentelemetry\-collector\-contrib\/.*\) v0\.135\.0/\1 v0.137.0/' go.mod * go mod tidy * update Elastic components * remove Elastic Trace processor In anticipation of #10378 * go get go.opentelemetry.io/[email protected] * go mod tidy * add changelog entry (cherry picked from commit 7ea323e) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md
pkoutsovasilis
added a commit
that referenced
this pull request
Oct 16, 2025
….137.0 (#10543) * feat: update OTel Collector components to v0.137.0 (#10391) * sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v0\.135.0/\1 v0.137.0/' go.mod * sed -i 's/\(go\.opentelemetry\.io\/collector.*\) v1\.41\.0/\1 v1.43.0/' go.mod * sed -i 's/\(github\.com\/open-telemetry\/opentelemetry\-collector\-contrib\/.*\) v0\.135\.0/\1 v0.137.0/' go.mod * go mod tidy * update Elastic components * remove Elastic Trace processor In anticipation of #10378 * go get go.opentelemetry.io/[email protected] * go mod tidy * add changelog entry (cherry picked from commit 7ea323e) # Conflicts: # NOTICE-fips.txt # NOTICE.txt # go.mod # go.sum # internal/pkg/otel/README.md * fix: resolve conflicts * fix: StartMockES * [otel] Add back elastictrace processor * fix: resolve gaps from rebasing * fix: unit-tests * fix: update beats to latest version on 8.19 * fix: otelconfig_test.go --------- Co-authored-by: Andrzej Stencel <[email protected]> Co-authored-by: Panos Koutsovasilis <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Removes the Elastic Trace processor from EDOT Collector, replacing all usages with its direct drop-in replacement, the Elastic APM processor.
Why is it important?
The Elastic Trace processor was deprecated and its code was removed in elastic/opentelemetry-collector-components#784.
Checklist
./changelog/fragmentsusing the changelog tool[ ] I have added an integration test or an E2E testDisruptive User Impact
This is a breaking change for users if they use the processor in their EDOT Collector configurations.
The fix is simple: replace
elastictracewithelasticapmin their configurations.How to test this PR locally